-
Notifications
You must be signed in to change notification settings - Fork 41
FMA: Interop Transaction Handling #249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
ce0c767
to
6d90225
Compare
The framing of two extremes is a great introduction to this topic |
Is there any consideration with the batcher? Did we add some new interop specific config? |
There could be a failure mode of disk growth given we have to index a lot of data. The solution seems to be pruning, which we will know the depth at which we can prune once we get the final expiry window |
Another more esoteric option for checks: |
@SozinM thanks for this idea! I think this is functionally equivalent to our decision to batch-evaluate these messages on a recurring timer. If the timer happened to match the block building time, you'd achieve the same effect, where transactions get evaluated in anticipation of the next block. Hooking it directly to block number sounds neat, but I wouldn't want to put validation in the way of our block building timing. And I totally agree regarding only validating on block's worth in advance. The nice thing is that we can statically determine if transactions are interop or not, so if there are 100k tx in the pool, we will only need to look over them once to identify the interop ones, and batch-check those (which would be a small subset of all possible tx) |
Takeaways from review meeting:
|
Not really, or at least not with respect to transaction ingress. We've identified that the Batcher should likely avoid publishing content beyond the cross unsafe head, which reduces the risk of remote chain invalidation affecting the batch, and that was called out in the Supervisor FMA and now is mentioned in some related task lists: ethereum-optimism/optimism#15175 |
Even with fairly aggressive estimates, a cloud host would not be strained by the Supervisor's disk usage: ethereum-optimism/optimism#14919 |
682a2b9
to
79a7498
Compare
No description provided.